Skip to content

Update oae.yml#1099

Merged
jamesaoverton merged 3 commits intomasterfrom
zhengj2007-patch-2
Apr 16, 2026
Merged

Update oae.yml#1099
jamesaoverton merged 3 commits intomasterfrom
zhengj2007-patch-2

Conversation

@zhengj2007
Copy link
Copy Markdown
Contributor

updated url of oae.owl
added oae-base.owl with url
updated issue tracker redirect url
added oae release version IRI redirects

updated url of oae.owl 
added oae-base.owl with url
updated issue tracker redirect url
added oae release version IRI redirects

This comment was marked as abuse.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Member

@jamesaoverton jamesaoverton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot made a complete mess of this PR, and I'm not going to waste my limited volunteer hours correcting it.

I'm happy to help humans learn how to update the PURL configs.

@jamesaoverton jamesaoverton marked this pull request as draft April 15, 2026 20:13
@zhengj2007
Copy link
Copy Markdown
Contributor Author

Hi @jamesaoverton I made the changes manually and used Copilot to review them to see how well it performed. Sorry for adding so many comments that made the pull request a bit messy. I’ve now hidden the Copilot comments.
I’d really appreciate it if you could review my changes again. Thanks!

@jamesaoverton
Copy link
Copy Markdown
Member

Ok. I'm done work for today, but I will review it tomorrow.

Comment thread config/oae.yml Outdated
Comment thread config/oae.yml Outdated
Comment thread config/oae.yml
Comment thread config/oae.yml
@jamesaoverton jamesaoverton dismissed stale reviews from matentzn and themself April 16, 2026 13:50

Thanks Nico. I addressed all your points.

@jamesaoverton jamesaoverton marked this pull request as ready for review April 16, 2026 13:51
@jamesaoverton jamesaoverton merged commit 10744ea into master Apr 16, 2026
1 check passed
@jamesaoverton jamesaoverton deleted the zhengj2007-patch-2 branch April 16, 2026 13:52
@zhengj2007
Copy link
Copy Markdown
Contributor Author

Thanks, @jamesaoverton @matentzn

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants